Skip to content

[Feature] Get rid of wget dependency#4468

Open
400Ping wants to merge 6 commits intoray-project:masterfrom
400Ping:feature/remove-wget
Open

[Feature] Get rid of wget dependency#4468
400Ping wants to merge 6 commits intoray-project:masterfrom
400Ping:feature/remove-wget

Conversation

@400Ping
Copy link
Contributor

@400Ping 400Ping commented Jan 30, 2026

Why are these changes needed?

KubeRay's health checks and the RayJob submitter's GCS wait loop used to rely on wget to perform HTTP requests. Slim Ray images (Ray 2.53+) do not include wget, which led to probe or job submission failures. These changes remove the wget dependency for Ray ≥ 2.53 so KubeRay works with slim images that only provide Ray (and Python). For Ray < 2.53, wget is still used where exec probes or the GCS wait are needed, since slim images are only released for 2.53+.

Change summary

  • RayCluster (head/worker): For Ray ≥ 2.53, use HTTP probes to /api/healthz (no wget). For Ray < 2.53, keep wget for exec-based health checks to the same endpoints.
  • RayJob submitter (K8s Job / Sidecar mode): For Ray ≥ 2.53, the "wait for GCS" loop uses a Python-based health check instead of wget. For Ray < 2.53, the existing wget-based wait is kept.
  • RayService worker: Readiness uses a single HTTP probe to /-/healthz on the serving port (Serve proxy health). Liveness uses node health (/api/healthz) to check.

Related issue number

Closes #3837

Checks

  • I've made sure the tests are passing.
  • Testing Strategy
    • Unit tests
    • Manual tests
    • This PR is not tested :(

Signed-off-by: 400Ping <jiekaichang@apache.org>
Signed-off-by: 400Ping <jiekaichang@apache.org>
Signed-off-by: 400Ping <jiekaichang@apache.org>
Signed-off-by: 400Ping <jiekaichang@apache.org>
@400Ping 400Ping marked this pull request as ready for review February 4, 2026 14:30
Copy link
Member

@win5923 win5923 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we need change to urllib.request for Ray < 2.53.0?
Since the slim images (without wget) are only being released for Ray 2.53+, I think older versions can keep using wget.

Signed-off-by: 400Ping <jiekaichang@apache.org>
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Copilot AI mentioned this pull request Feb 5, 2026
4 tasks
Signed-off-by: 400Ping <jiekaichang@apache.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Get rid of wget dependency

2 participants